Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix users getting stuck on welcome screen #7726

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

rablador
Copy link
Contributor

@rablador rablador commented Feb 26, 2025

Why this needs to be done

Reproduce:

  • Create new account
  • Purchase time in welcome view
  • Close app (do not click Start using the app
  • Start app again
  • Now the user is back on the Welcome view and needs to purchase more time to continue.

It can be solved by logging out and back in, but this is not a good first experience.

What needs to be done

isShownOnboarding in AppPreferences should be set to true as soon as the user has been past the Welcome view.

Check if the user has time before presenting WelcomeView.

Acceptance criteria

Reproducing the above steps should lead the user to the main view instead of back to the welcome view.


This change is Reviewable

@rablador rablador added the iOS Issues related to iOS label Feb 26, 2025
@rablador rablador self-assigned this Feb 26, 2025
Copy link

linear bot commented Feb 26, 2025

Copy link
Contributor

@buggmagnet buggmagnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: 0 of 1 files reviewed, all discussions resolved

Copy link
Collaborator

@pinkisemils pinkisemils left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@rablador rablador force-pushed the new-user-can-get-stuck-on-welcome-screen-ios-1084 branch from ec6732a to dc95ff5 Compare March 3, 2025 14:34
Copy link
Contributor

@buggmagnet buggmagnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@rablador rablador force-pushed the new-user-can-get-stuck-on-welcome-screen-ios-1084 branch from dc95ff5 to 683b7e3 Compare March 3, 2025 14:41
@rablador rablador merged commit b132688 into main Mar 3, 2025
11 checks passed
@rablador rablador deleted the new-user-can-get-stuck-on-welcome-screen-ios-1084 branch March 3, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iOS Issues related to iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants